Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

linux: Add AT_SYSINFO_EHDR constant #3027

Merged
merged 1 commit into from
Dec 11, 2022
Merged

Conversation

Phantomical
Copy link
Contributor

When passed to getauxval(3) this is used to retrieve the base address of the vdso mapped into the current process. It is defined the architecture specific auxvec.h headers but it is defined to the same value in all of them so I've put it in src/unix/linux_like/linux/mod.rs instead of in the arch submodule.

@rustbot
Copy link
Collaborator

rustbot commented Dec 5, 2022

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @JohnTitor (or someone else) soon.

Please see the contribution instructions for more information.

@JohnTitor
Copy link
Member

Thanks! @bors r+

@bors
Copy link
Contributor

bors commented Dec 11, 2022

📌 Commit ce9ab41 has been approved by JohnTitor

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Dec 11, 2022

⌛ Testing commit ce9ab41 with merge 22eaf09...

bors added a commit that referenced this pull request Dec 11, 2022
linux: Add AT_SYSINFO_EHDR constant

When passed to [`getauxval(3)`][2] this is used to retrieve the base address of the vdso mapped into the current process. It is defined the architecture specific `auxvec.h` headers but [it is defined to the same value in all of them][1] so I've put it in `src/unix/linux_like/linux/mod.rs` instead of in the arch submodule.

[1]: https://sourcegraph.com/search?q=context:global+repo:%5Egithub%5C.com/torvalds/linux%24+%23define+AT_SYSINFO_EHDR&patternType=standard&sm=1
[2]: https://man7.org/linux/man-pages/man3/getauxval.3.html
@bors
Copy link
Contributor

bors commented Dec 11, 2022

💔 Test failed - checks-actions

@JohnTitor
Copy link
Member

@bors retry

@bors
Copy link
Contributor

bors commented Dec 11, 2022

⌛ Testing commit ce9ab41 with merge 6a5c07f...

@bors
Copy link
Contributor

bors commented Dec 11, 2022

☀️ Test successful - checks-actions, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13, checks-cirrus-freebsd-14
Approved by: JohnTitor
Pushing 6a5c07f to master...

@bors bors merged commit 6a5c07f into rust-lang:master Dec 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants